Skip to content

Claude/fix dropdown class options 011 c up hr laq eg62k gyh sd8e g - #6

Merged
RobGruhl merged 2 commits into
mainfrom
claude/fix-dropdown-class-options-011CUpHrLaqEG62kGyhSd8eG
Nov 5, 2025
Merged

Claude/fix dropdown class options 011 c up hr laq eg62k gyh sd8e g#6
RobGruhl merged 2 commits into
mainfrom
claude/fix-dropdown-class-options-011CUpHrLaqEG62kGyhSd8eG

Conversation

@RobGruhl

@RobGruhl RobGruhl commented Nov 5, 2025

Copy link
Copy Markdown
Owner

No description provided.

Issue: Balcor had the 'machine' class which is intended for robots/AI characters,
but Balcor is an organic Nerind'ar engineer.

Changes:
- Replaced 'machine' class with 'tech' class in Balcor's classBoards
- Removed redundant 'impervious' skill specification (already provided by support class)
- Balcor now has access to engineering-appropriate skills: engineering, regulate,
  hackAndGrab, assist, camouflage, hardToHit, and distraction

This makes thematic sense for Balcor's character description as a
"Skilled Nerind'ar engineer with modified prosthetic arm."
Issue: Characters like Balcor and Wade could only see 3-4 class options
in the dropdown, when they should be able to choose from all available classes.

Root Cause: The resolveAvailableClasses() function was limiting characters
to only the classes listed in their classBoards array, regardless of whether
they were machine-restricted characters or not.

Changes:
- Added machine-only character detection (checks if all classBoards are machine/machineTech)
- Machine characters (Hopper, XL) still get only their restricted classes
- Non-machine characters now see ALL global classes (all 14 classes)
- Character-specific skill customizations from classBoards are merged into global classes
- Characters without classBoards continue to get all classes (unchanged)

Result:
- Balcor now sees all 14 classes instead of just 3
- Wade now sees all 14 classes instead of just 4
- Balcor's custom Marine/Support/Tech skill lists are preserved
- Wade's custom skill lists for his classes are preserved
- Machine characters (Hopper, XL) remain restricted to machine classes only
@RobGruhl
RobGruhl merged commit d75f5a4 into main Nov 5, 2025
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants